Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

dummy PR for tests #403

Closed
wants to merge 1 commit into from
Closed

dummy PR for tests #403

wants to merge 1 commit into from

Conversation

slava77
Copy link
Contributor

@slava77 slava77 commented May 17, 2024

No description provided.

@slava77
Copy link
Contributor Author

slava77 commented May 17, 2024

/run cmssw 27

@slava77 slava77 changed the title space in readme for tests dummy PR for tests May 17, 2024
@slava77 slava77 marked this pull request as draft May 17, 2024 20:58
Copy link

There was a problem while building and running with CMSSW. The logs can be found here.

@slava77
Copy link
Contributor Author

slava77 commented May 20, 2024

/run cmssw 28

Copy link

The PR was built and ran successfully with CMSSW. Here are some plots.

OOTB All Tracks
Efficiency and fake rate vs pT, eta, and phi

The full set of validation and comparison plots can be found here.

@VourMa
Copy link
Contributor

VourMa commented May 21, 2024

Also to highlight the main motivation for these changes:
image

@ariostas ariostas closed this Dec 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants